-
Notifications
You must be signed in to change notification settings - Fork 41
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Refactor shared logic for generating package.json scripts for FF and FE #2386
Conversation
🦋 Changeset detectedLatest commit: 2e8cf5e The changes in this PR will be included in the next version bump. This PR includes changesets to release 7 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
…x-tools into package-config-common
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Refactoring to remove duplication looks good.
writer snapshots not impacted by the refactoring ✅
Tested moved ✅
changeset ✅
…x-tools into package-config-common
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @kjose90
- Consolidates package script creation code
- No updates to snapshots indicates functionally identical
- Tests added to cover new code
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Re-approving. Thanks
Quality Gate passedIssues Measures |
…on-CAP-Fiori-generation-into-a-CAP-project-folder * origin/main: chore: apply latest changesets feat(ui-prompting): rename ui prompting examples package from `@sap-ux/prompting-ui` to `@sap-ux-private/ui-prompting-examples` (#2415) chore: apply latest changesets fix: adjust condition for dynamic header field content (#2412) chore: apply latest changesets feat(ui-prompting): support translation input fields for prompts (#2400) chore: apply latest changesets Renamed the skipLiveScript parameter to addLiveConfig (#2405) chore: apply latest changesets fix: various ui improvements and bug fix (#2410) chore: apply latest changesets Refactor shared logic for generating package.json scripts for FF and FE (#2386) chore: apply latest changesets feat: enhance quick action with the template (#2388) chore: apply latest changesets fix(cpe): application mode after reload (#2389)
…FE (#2386) * refactoring shared logic for adding package json scriots for FF and FE * add changeset * add docs * pnpm recursive install * revert translations mockOnlyWarning * fix sonar issues * resolving conflictas after merge * revise export of function * change name to getPackageScripts * fix test name * test fix * add lint ignore escape chars * add lint ignore escape chars * es lint * add a comment explaining not to remove escape chars from command --------- Co-authored-by: Austin Devine <[email protected]>
#2320
package.json
scripts intofiori-generator-shared
, ensuring that both Fiori Elements and Freestyle applications use a common, reusable implementation.